Update logits_process.py docstrings to clarify penalty and reward cases (attempt #2)#26784
Update logits_process.py docstrings to clarify penalty and reward cases (attempt #2)#26784ArthurZucker merged 2 commits intohuggingface:mainfrom larekrow:patch-1
Conversation
|
Could you run |
|
Hey @ArthurZucker, I tried to run Anyway I thought I should not be encountering this in the first place since I only modified docstrings in a single file. So, I ran Even |
|
Hey! we have "ruff>=0.0.241,<=0.0.259" |
|
Thanks for the guidance @ArthurZucker! The test has passed 😄 |
|
Cool thanks for the contribution |
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. |
…es (attempt #2) (huggingface#26784) * Update logits_process.py docstrings + match arg fields to __init__'s * Ran `make style`
This PR fixes point 3 of #25970 by clarifying the penalty and reward cases for RepetitionPenaltyLogitsProcessor and EncoderRepetitionPenaltyLogitsProcessor within the docstrings.
PR #26129 was the original copy, but I have accidentally deleted my repo that submitted the PR, so I cannot reopen that PR 🙇
@gante, for your review please.